Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

put Linux build files inside a folder before zipping #870

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AlbyIanna merged 2 commits into main from linux-build-inside-folder
Mar 1, 2022

Conversation

@AlbyIanna
Copy link
Contributor

@AlbyIanna AlbyIanna commented Feb 28, 2022

Motivation

When you download the Arduino IDE in zip format for Linux and extract it, a lot of the files will be extracted in that folder. If a user does that in the Downloads folder, they will find it overflooded with files.

Change description

Reintroduce a function in the packager script to add a folder on top of the zip build, but only when on Linux.

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@AlbyIanna AlbyIanna changed the base branch from main to custom-update-url February 28, 2022 12:41
Copy link
Contributor

fstasi commented Feb 28, 2022

@per1234 can you test if appImage is still working on linux? I' not 100% sure the appImage will do NOT check the zip sha as well

per1234 reacted with eyes emoji

Base automatically changed from custom-update-url to main February 28, 2022 14:04
@per1234 per1234 added os: linux Specific to Linux operating system topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement labels Mar 1, 2022
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "Linux_X86-64_zip" build artifact has the perfect structure (files in a folder instead of in the root of the .zip).

As requested, I did a test of the IDE auto-update behavior after this change. I produced the following channel file to allow me to test this build as the update target in addition to testing it as the update base:

version: 99.0.1
files:
 - url: arduino-ide_2.0.0-rc4-snapshot-85f5d6f_Linux_64bit.AppImage
 sha512: 1YE6Y5/MegOgE9NlrO/PrPACPMrpC/5cuVJUOgY8xsYn+KcFpVNYtqKldMVPUEJKkEw1aBvlaPp0BzFeo4rYQg==
 size: 246645528
 blockMapSize: 258989
path: arduino-ide_2.0.0-rc4-snapshot-85f5d6f_Linux_64bit.AppImage
sha512: 1YE6Y5/MegOgE9NlrO/PrPACPMrpC/5cuVJUOgY8xsYn+KcFpVNYtqKldMVPUEJKkEw1aBvlaPp0BzFeo4rYQg==
releaseDate: 2022年02月28日T03:10:27.201Z

It worked perfectly.

Thanks Alberto!

AlbyIanna reacted with rocket emoji
@AlbyIanna AlbyIanna merged commit 2a27a14 into main Mar 1, 2022
@AlbyIanna AlbyIanna deleted the linux-build-inside-folder branch March 1, 2022 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@per1234 per1234 per1234 approved these changes

+1 more reviewer

@fstasi fstasi fstasi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

os: linux Specific to Linux operating system topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /